Skip to content

Conversation

kbenzie
Copy link
Contributor

@kbenzie kbenzie commented Aug 8, 2025

Automated changes by create-pull-request GitHub action

RossBrunton and others added 3 commits August 8, 2025 00:44
If a buffer set fails then the test now exits early. This ensures that
the device isn't asked to run a kernel on invalid memory.
The problem was `OpenCL_FOUND` was getting set to `TRUE` on the second
invocation of the module because `find_package` was finding the
`FetchContent` clone in `SOURCE_TREE/build/__deps`.

We use `OpenCL_FOUND` to specifically refer to external system-wide
installs, so use `NO_CMAKE_PACKAGE_REGISTRY` to make sure we don't find
FetchContent clones.

Also not related to the fix here but re-order `FetchContent_Declare`
`FetchContent_GetProperties`, the `Declare` should come first and we
`MakeAvailable` should not be inside the `if`, we only check if it's
populated to print the message.

Closes: intel/llvm#19737

Signed-off-by: Sarnie, Nick <[email protected]>
@kbenzie kbenzie requested a review from a team as a code owner August 8, 2025 00:44
Copy link
Contributor

github-actions bot commented Aug 8, 2025

Unified Runtime -> intel/llvm Repo Move Notice

Information

The source code of Unified Runtime has been moved to intel/llvm under the unified-runtime top-level directory,
all future development will now be carried out there. This was done in intel/llvm#17043.

The code will be mirrored to oneapi-src/unified-runtime and the specification will continue to be hosted at oneapi-src.github.io/unified-runtime.

The contribution guide will be updated with new instructions for contributing to Unified Runtime.

PR Migration

All open PRs including this one will be marked with the auto-close label and shall be automatically closed after 30 days.

Should you wish to continue with your PR you will need to migrate it to intel/llvm.
We have provided a script to help automate this process.

If your PR should remain open and not be closed automatically, you can remove the auto-close label.


This is an automated comment.

@aarongreig aarongreig merged commit e3e405d into main Aug 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants